Skip to content

33515 updating the old COOPS outputs to the modern templates#4496

Merged
vysakh-menon-aot merged 3 commits into
bcgov:mainfrom
vysakh-menon-aot:feature/33515
Jun 17, 2026
Merged

33515 updating the old COOPS outputs to the modern templates#4496
vysakh-menon-aot merged 3 commits into
bcgov:mainfrom
vysakh-menon-aot:feature/33515

Conversation

@vysakh-menon-aot

@vysakh-menon-aot vysakh-menon-aot commented Jun 15, 2026

Copy link
Copy Markdown
Collaborator

Issue #: /bcgov/entity#33515

Description of changes:
image

image image

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the lear license (Apache 2.0).

@kialj876 kialj876 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey Vysakh, can we condense the templates? It looks like BC and CP are almost the exact same now. All my comments below are to do with that

Comment thread legal-api/report-templates/template-parts/addresses.html
Comment thread legal-api/report-templates/template-parts/directors.html
Comment thread legal-api/report-templates/annualReport.html
Comment thread legal-api/report-templates/changeOfAddress.html
Comment thread legal-api/report-templates/changeOfDirectors.html
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
35.2% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@kialj876 kialj876 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the updates!

@vysakh-menon-aot vysakh-menon-aot merged commit 97b997f into bcgov:main Jun 17, 2026
7 of 8 checks passed
@vysakh-menon-aot vysakh-menon-aot deleted the feature/33515 branch June 17, 2026 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants